Skip to content
This repository was archived by the owner on Sep 9, 2024. It is now read-only.

feat: remove required name in media_library config#234

Merged
KaneFreeman merged 1 commit intoStaticJsCMS:mainfrom
mildred:fix/media-library
Dec 14, 2022
Merged

feat: remove required name in media_library config#234
KaneFreeman merged 1 commit intoStaticJsCMS:mainfrom
mildred:fix/media-library

Conversation

@mildred
Copy link
Contributor

@mildred mildred commented Dec 14, 2022

name is only required to configure an external media library, but the media_library config can be used without a name to configure the max file size for example.

See in core/src/mediaLibrary.ts the isExternalMediaLibraryConfig function.

name is only required to configure an external media library, but the
media_library config can be used without a name to configure the max
file size for example.

See in `core/src/mediaLibrary.ts` the `isExternalMediaLibraryConfig`
function.
@netlify
Copy link

netlify bot commented Dec 14, 2022

Deploy Preview for staticjscms canceled.

Name Link
🔨 Latest commit 96bfe1b
🔍 Latest deploy log https://app.netlify.com/sites/staticjscms/deploys/639a2546dc146c00083b6e25

@mildred
Copy link
Contributor Author

mildred commented Dec 14, 2022

Do you have an idea how I can build the dist js file so I can test this?

edit: oops I forgot to cd core before running yarn...

@KaneFreeman KaneFreeman changed the title Remove required name in media_library config feat: remove required name in media_library config Dec 14, 2022
@KaneFreeman KaneFreeman merged commit ee8fcf4 into StaticJsCMS:main Dec 14, 2022
@mildred mildred deleted the fix/media-library branch December 15, 2022 08:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants